Skip to content

fix: steady-state reconcile write churn#21

Open
PKizzle wants to merge 1 commit into
jr42:mainfrom
PKizzle:fix/reconcile-churn
Open

fix: steady-state reconcile write churn#21
PKizzle wants to merge 1 commit into
jr42:mainfrom
PKizzle:fix/reconcile-churn

Conversation

@PKizzle

@PKizzle PKizzle commented May 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • skip DynamicPrefix status updates when the calculated status is unchanged
  • compare desired and current Cilium pool/BGP specs semantically before writing
  • update Service last-sync only when managed annotations actually change
  • raise the chart's default memory limit from 128Mi to 256Mi
  • add steady-state second-reconcile regression coverage for controller writes

Testing

  • go test ./internal/controller -count=1
  • make lint
  • make helm-lint

Fixes #16

@PKizzle PKizzle changed the title Fix steady-state reconcile write churn fix: steady-state reconcile write churn May 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reconcile loop: poolsync ↔ dynamicprefix writes 7+/s to apiserver indefinitely (post-Apr-12 main)

1 participant